Skip to content

Feature/branch#1

Open
Arturgouveia1970 wants to merge 5 commits intoIbaslogic:mainfrom
Arturgouveia1970:feature/branch
Open

Feature/branch#1
Arturgouveia1970 wants to merge 5 commits intoIbaslogic:mainfrom
Arturgouveia1970:feature/branch

Conversation

@Arturgouveia1970
Copy link

added some styling

@yusufnviiri
Copy link

yusufnviiri commented Jun 16, 2022

Hi, @Arturgouveia1970 great work done on this project 👍.

  • 🤜 GitHub flow 💯 followed.
  • 🤜 Descriptive commit messages.
  • 🤜 clean Code.

However, I think my comments in this section can make your code better ♻️

  • Kindly Add Linter checks to your project. This will make your work more professional and set to standard.
  • You will realize that when you add the To-Do item, the input field is not cleared after submission, Kindly fix that.
  • One of the project requirements was to implement the Nav bar section using react-router, this section is missing in your project, Kindly add it.

**Happy coding 👍 **

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants